Skip to content

drivers: misc: Add Low-voltage detection (LVD) driver support for RX130 #93694

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

quytranpzz
Copy link
Contributor

Introduce low-voltage detection driver support on RX130
The voltage detection circuit (LVD) monitors the voltage level input to the VCC or external pin using a program.
Upon detecting a voltage level transition (either upward or downward) past the configured threshold, the system can generate a reset or an interrupt, enabling appropriate protective or recovery actions.

Copy link

github-actions bot commented Jul 25, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_renesas DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Jul 25, 2025
@duynguyenxa duynguyenxa added the platform: Renesas RX Renesas Electronics Corp, RX label Jul 25, 2025
@quytranpzz quytranpzz force-pushed the renesas_rx_lvd_support branch 6 times, most recently from 6018f8c to c11e903 Compare July 28, 2025 04:46
@quytranpzz quytranpzz marked this pull request as ready for review August 5, 2025 09:51
@zephyrbot zephyrbot added area: Samples Samples platform: Renesas RA Renesas Electronics Corporation, RA area: RX area: Pinctrl labels Aug 5, 2025
Comment on lines 15 to 19
config RENESAS_RX_LVD_INIT_PRIORITY
int "RX LVD initialization priority"
default KERNEL_INIT_PRIORITY_DEVICE
help
System initialization priority for RX LVD driver.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be wrapped by RENESAS_RX_LVD

Suggested change
config RENESAS_RX_LVD_INIT_PRIORITY
int "RX LVD initialization priority"
default KERNEL_INIT_PRIORITY_DEVICE
help
System initialization priority for RX LVD driver.
if RENESAS_RX_LVD
config RENESAS_RX_LVD_INIT_PRIORITY
int "RX LVD initialization priority"
default KERNEL_INIT_PRIORITY_DEVICE
help
System initialization priority for RX LVD driver.
endif # RENESAS_RX_LVD

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's done

* SPDX-License-Identifier: Apache-2.0
*/

/* Local Includes */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please drop this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed this

Update commit ID for hal_renesas for adding LVD support

Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
Add header file to support Renesas LVD APIs on Zephyr

Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
Introduce Low/Programmable voltage detection driver for Renesas RX
which using r_lvd_rx from RDP. The target is to monitor the voltage
level input to Vcc/CMPA2 pin using a program

Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
Add LVD node support on RX130

Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
Add a simple sample for LVD driver on RSK-RX130-512kb

Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
@quytranpzz quytranpzz force-pushed the renesas_rx_lvd_support branch from c11e903 to d012894 Compare August 15, 2025 10:45
@github-actions github-actions bot removed manifest manifest-hal_renesas DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Aug 15, 2025
Copy link

Copy link
Contributor

@thenguyenyf thenguyenyf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Pinctrl area: RX area: Samples Samples platform: Renesas RA Renesas Electronics Corporation, RA platform: Renesas RX Renesas Electronics Corp, RX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants